-
Notifications
You must be signed in to change notification settings - Fork 8
chore: update Angular and igniteui-angular to 21.0.0 #1463
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: Lipata <[email protected]>
Co-authored-by: Lipata <[email protected]>
|
@copilot, if you create an empty project with Angular CLI, using the command |
I've analyzed the differences between a fresh Angular 21.0 Key Intentional Differences (Correct as-is):The igx-ts-legacy template is designed for legacy NgModule-based apps, so these differences are expected:
Important Angular 21.0 Changes from Changelog:
Potential Improvements for Better Angular 21 Alignment:While our current versions are correct, we could consider these TypeScript config enhancements to match modern Angular 21 best practices:
However, since this is the legacy template, maintaining the current configuration is reasonable. The igx-ts (standalone) template would be more appropriate for these modern features. Recommendation: The current changes are sufficient. The template is correctly configured for Angular 21.0 with legacy NgModule architecture. Would you like me to add the stricter TypeScript compiler options, or keep the template as-is? |
|
@copilot, I have the same question for the |
I've compared the igx-ts (standalone) template with a fresh Angular 21.0 Key Differences Found:1. Build System:
2. Testing:
3. TypeScript Config:
4. Package.json:
What's Already Good:
Recommendation:The igx-ts template is well-configured for Angular 21. The differences are intentional choices for stability (Karma vs Vitest, legacy builder vs new builder). Optional enhancement: Could update No changes required - the template is production-ready for Angular 21.0. |
|
@copilot let's try and change 1,2 and 3. |
…mpiler options, use Bundler moduleResolution, and preserve module Co-authored-by: Lipata <[email protected]>
Done! Updated TypeScript configurations for both templates in commit 769090c: Changes applied to both igx-ts and igx-ts-legacy:
Build and lint pass successfully. Templates are now fully aligned with Angular 21 best practices. |
|
@copilot can you also add @angular/build:application |
…tion builder Co-authored-by: Lipata <[email protected]>
Done in commit eb245d2. Added Changes made:
Note: Kept |
Update Angular and igniteui-angular to version 21.0.0
Following the pattern from PR #1382 (which updated from 19.0 to 19.1), this PR updates the templates to use Angular 21.0 and igniteui-angular 21.0.
All Tasks Completed ✅
Updated dependencies:
TypeScript Configuration Updates (Angular 21 alignment):
Both igx-ts and igx-ts-legacy templates:
noImplicitOverride,noPropertyAccessFromIndexSignature,noImplicitReturns,noFallthroughCasesInSwitch,isolatedModulesmoduleResolutionfrom "node" to "Bundler" (igx-ts-legacy)modulefrom "ES2022" to "preserve" for better compatibility with Angular 21 build systemBuild System Updates:
Both igx-ts and igx-ts-legacy templates:
@angular/buildpackage as devDependencyangular.jsonto use@angular/build:applicationfor build targetangular.jsonto use@angular/build:dev-serverfor serve targetFiles changed (9):
packages/igx-templates/constants.ts- Updated igniteui-angular version constantpackages/igx-templates/igx-ts/projects/_base/files/package.json- Updated to Angular 21.0.0, added @angular/buildpackages/igx-templates/igx-ts/projects/_base/files/tsconfig.json- Added isolatedModules, changed module to preservepackages/igx-templates/igx-ts/projects/_base/files/angular.json- Updated builders to @angular/buildpackages/igx-templates/igx-ts/projects/side-nav-auth/files/package.json- Updated to Angular 21.0.0, added @angular/buildpackages/igx-templates/igx-ts-legacy/projects/_base/files/package.json- Updated to Angular 21.0.0, added @angular/buildpackages/igx-templates/igx-ts-legacy/projects/_base/files/tsconfig.json- Added strict options, Bundler moduleResolution, preserve modulepackages/igx-templates/igx-ts-legacy/projects/_base/files/angular.json- Updated builders to @angular/buildpackages/igx-templates/igx-ts-legacy/projects/side-nav-auth/files/package.json- Updated to Angular 21.0.0, added @angular/buildVerification:
✅ Build passes successfully
✅ Lint passes with only pre-existing warnings
✅ Code review completed with positive feedback
✅ Security checks passed (no code changes to analyze)
✅ yarn.lock remains unchanged
✅ TypeScript configs aligned with Angular 21 best practices
✅ Using modern @angular/build system
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.